Desafio 03 feito em javascript por @joseildoandrade12 #1157
Conversation
There was a problem hiding this comment.
@joseildoandrade12 a solução tem que permitir ao usuário selecionar o intervalo de números a ser analisado pelo programa e não analisar apenas o intervalo entre 1 e 100. Ajusde por favor a sua solução.
There was a problem hiding this comment.
@joseildoandrade12 seu programa não funciona coforme necessário. Antes de mandar o PR teste na sua máquina e assegure que está fazendo o que está fazendo. Seu programa não está pedindo o intervalo de números ao usuário. Por favor não submita PR para review sem isso estar funcionando na sua máquina. Obrigado!
| ## Desafio 03: Números Palíndromos | ||
|
|
||
| Um palíndromo é uma palavra ou expressão que pode ser lida da esquerda para a direita ou da direita para a esquerda mantendo o mesmo significado. Um exemplo de palíndromo é a palavra “reviver”. | ||
|
|
There was a problem hiding this comment.
Seu readme não contém as instruções de como rodar seu programa, muito menos os requisitos necessários.
Adicione essas informações nele, sem isso não consigo testar sua solução.
There was a problem hiding this comment.
tem algum exemplo de como eu possa fazer essa parte? algum README de inspiração? não sou bom com o readme
| ## Teste e execução | ||
|
|
||
| Para testar a solução, é necessário ter o [Node.js](https://nodejs.org/) instalado em sua máquina. | ||
|
|
There was a problem hiding this comment.
Poxa cara, eu te mandei um bom exemplo e você modificou ele tirando a parte mais importante...
O texto tá igual na parte de teste e execução, só ajustaste o link do git.
Mas a principal linha tu tirou:
Faça um clone do repositório para a máquina local, e rode o arquivo no node, passando os números como argumentos:
|
|
||
| Para testar a solução, é necessário ter o [Node.js](https://nodejs.org/) instalado em sua máquina. | ||
|
|
||
| ```bash |
There was a problem hiding this comment.
Com isso agora fica solto as suas instruções de bash a seguir.
Qual é a versão do Node que vc tá usando?
Como que faz o processo pra rodar sua solução?
Essas coisas precisam estar no Readme.
Meu comentário inicial fala dos requisitos e das instruções.
Você botou pela metade os requisitos e menos da metade das instruções...
Trabalhe em melhorar o Readme com as suas próprias palavras.
Como estudo, olhe o readme do próprio repositório dos desafios.
Compare ele com as instruções que você fez.
Imagine se a mesma abordagem que a sua fosse usada no repositório dos desafios e faça a seguinte pergunta:
Ficou mais fácil?
There was a problem hiding this comment.
Perdão pelo erro, irei ajustar assim que possível e tentarei deixar de forma mais clara possível de como utilizar. Agradeço por apontar os erros!
| Faça um clone do repositório para a sua máquina: | ||
|
|
||
| ```bash | ||
| $ git clone [email protected]:OsProgramadores/op-desafios.git |
There was a problem hiding this comment.
Essa url é para clone do repositório do os programadores, não do seu fork.
| Acesse o diretório com: | ||
|
|
||
| ```bash | ||
| $ cd .\op-desafios\ |
There was a problem hiding this comment.
A forma como você coloca os comandos é dependente da plataforma.
Ao executar este comando em um terminal linux ele não vai entrar no diretório op-desafios.
Onde você está desenvolvendo, você precisa colocar esse . antes da barra para acessar o subdiretório?
| Rode o arquivo no node da seguinte forma: | ||
|
|
||
| ```bash | ||
| $ node .\desafio-03\joseildoandrade12\javascript script.js |
There was a problem hiding this comment.
Aqui você instrui a executar algo que não está disponível no repositório base antes de ser mergeado.
Também tem a mesma questão do . antes de passar o caminho.
De brinde tem o detalhe do path estar quebrado, já que tem um espaço em branco em javascript script.
Você não executou essas instruções que colocaste aqui, né?
| $ node .\desafio-03\joseildoandrade12\javascript script.js | ||
| ``` | ||
|
|
||
| Após iniciar o arquivo irá aparecer as seguintes perguntas: |
There was a problem hiding this comment.
A partir deste ponto não é possível validar, visto que as instruções atuais não permitiram executar o seu código.
|
Devido ao desprezo deste usuário para com o meu tempo, não revisarei mais nenhum PR do mesmo. |
No description provided.